New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improving the RETURN and its docs on the apt_repository module #79658
Conversation
…he filename parameter
Thanks for your Ansible docs contribution! We talk about Ansible documentation on matrix at #docs:ansible.im and on libera IRC at #ansible-docs if you ever want to join us and chat about the docs! We meet there on Tuesdays (see the Ansible calendar) and welcome additions to our weekly agenda items - scroll down to find the upcoming agenda and add a comment to put something new on that agenda. |
The test
|
The test
|
here it goes @bcoca |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just create a new if
on top of the for loop
Any news? @nitzmahone |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs portion LGTM
@bcoca can this be merged now? |
The existing tests for this module are ... less than fantastic (for a number of reasons), but I added a few just to at least validate the shape of the result for non-check-mode. So long as they pass CI, I'll hit merge. Thanks! |
SUMMARY
Improving docs on how we generate the default value of the filename parameter of the apt_repository module
Fixes #79306
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/apt_repository.py